Skip to content

Conversation

DerEchtePilz
Copy link
Member

@DerEchtePilz DerEchtePilz commented Feb 17, 2025

TODO:

  • Update the setup pages to mention commandapi-paper-core, commandapi-paper-shade, commandapi-spigot-core and comandapi-spigot-shade instead of commandapi-bukkit-core and commandapi-bukkit-shade
  • Update setup info to mention CommandAPIPaperConfig/CommandAPISpigotConfig instead of CommandAPIBukkitConfig
  • Update Kotlin DSL intro to use the preferences switch
  • Update argument pages
    • FloatRangeArgument -> DoubleRangeArgument
    • PlayerArgument/OfflinePlayerArgument -> PlayerProfileArgument
    • AsyncOfflinePlayerArgument -> AsyncPlayerProfileArgument
    • Chat argument types are now named the same on each platform
    • ChatArgument returns a SignedMessage on Paper
    • BlockStateArgument now returns BlockState instead of BlockData
    • Update the page about cast types (also safe_suggestions.md)
  • Update config page
  • Update info on registering/unregistering commands since unregistering isn't necessary anymore if a command should be replaced
  • Update code references in docs source

@willkroboth willkroboth changed the base branch from master to ver/11.0.0-SNAPSHOT September 7, 2025 15:05
@DerEchtePilz DerEchtePilz marked this pull request as ready for review September 14, 2025 06:08
Copy link
Contributor

@willkroboth willkroboth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The separate modules in reference-code look good! It looks like the only compile issues are from the test toolkit, which I will fix in #13.

- Functions page needs a rewrite
- The IntegerRange/DoubleRange argument page needs a new screenshot
- Apparently there might be some rendering issues with the diff on the upgrade page to 11.0.0?
Also might include a fix for the diff language not being detected correctly
Copy link
Contributor

@willkroboth willkroboth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this comment is still unresolved.

Copy link
Contributor

@willkroboth willkroboth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. I guess there are still some open issues as mentioned on Discord, but those can be addressed later since they're not critical to the main documentation release.

@DerEchtePilz DerEchtePilz changed the base branch from ver/11.0.0-SNAPSHOT to master October 4, 2025 15:05
@DerEchtePilz DerEchtePilz merged commit 0e83059 into master Oct 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants